gtk/popover: Use gtk_popover_present() instead of going via GtkNative
authorJonas Ådahl <jadahl@gmail.com>
Fri, 4 Dec 2020 08:20:50 +0000 (09:20 +0100)
committerJonas Ådahl <jadahl@gmail.com>
Mon, 7 Dec 2020 19:37:29 +0000 (20:37 +0100)
This makes it more explicit that managers of popovers make it "present".

22 files changed:
demos/gtk-demo/demo3widget.c
demos/gtk-demo/suggestionentry.c
docs/reference/gtk/gtk4-sections.txt
gtk/gtkcolorswatch.c
gtk/gtkcolumnviewtitle.c
gtk/gtkcombobox.c
gtk/gtkdropdown.c
gtk/gtkemojichooser.c
gtk/gtkentrycompletion.c
gtk/gtkfilechooserwidget.c
gtk/gtklabel.c
gtk/gtklayoutmanager.c
gtk/gtkmenubutton.c
gtk/gtkplacessidebar.c
gtk/gtkplacesviewrow.c
gtk/gtkpopover.c
gtk/gtkpopover.h
gtk/gtkpopovermenubar.c
gtk/gtkscalebutton.c
gtk/gtktext.c
gtk/gtktextview.c
gtk/gtktreeview.c

index 3a309927e8f09a4a3633437d478481bf64315a42..7a13f698c4987e326414c2e167dee68907927138 100644 (file)
@@ -97,9 +97,9 @@ demo3_widget_size_allocate (GtkWidget *widget,
 
   /* Since we are not using a layout manager (who would do this
    * for us), we need to allocate a size for our menu by calling
-   * gtk_native_check_resize().
+   * gtk_popover_present().
    */
-  gtk_native_check_resize (GTK_NATIVE (self->menu));
+  gtk_popover_present (GTK_POPOVER (self->menu));
 }
 
 static void
index a0a926979f2c95ab0c37b717a5c0495e8025b325..8a0b3aa0825076a2d5292f217f64635fe8d5ebdc 100644 (file)
@@ -474,7 +474,7 @@ suggestion_entry_size_allocate (GtkWidget *widget,
   gtk_widget_set_size_request (self->popup, gtk_widget_get_allocated_width (GTK_WIDGET (self)), -1);
   gtk_widget_queue_resize (self->popup);
 
-  gtk_native_check_resize (GTK_NATIVE (self->popup));
+  gtk_popover_present (GTK_POPOVER (self->popup));
 }
 
 static gboolean
index f28400f7951be9328e9085acf6fb3c829d1607e1..8fea2b3c53e27432c95d8a596e404ee9fc67828d 100644 (file)
@@ -6112,6 +6112,7 @@ GtkPopover
 gtk_popover_new
 gtk_popover_popup
 gtk_popover_popdown
+gtk_popover_present
 gtk_popover_set_child
 gtk_popover_get_child
 gtk_popover_set_pointing_to
index 7addf913d39aceb0fe40f75d03962bb06053a492..a451fd827f074f55142271d560adf29c3edb88ac 100644 (file)
@@ -322,7 +322,7 @@ swatch_size_allocate (GtkWidget *widget,
                             }, -1);
 
   if (swatch->popover)
-    gtk_native_check_resize (GTK_NATIVE (swatch->popover));
+    gtk_popover_present (GTK_POPOVER (swatch->popover));
 }
 
 static void
index c76c8a9a8b391484aca6a611cdd9029ce8c5c284..160b1b7604085a471e0cc6a13feb617429157cb3 100644 (file)
@@ -144,7 +144,7 @@ gtk_column_view_title_size_allocate (GtkWidget *widget,
     }
 
   if (self->popup_menu)
-    gtk_native_check_resize (GTK_NATIVE (self->popup_menu));
+    gtk_popover_present (GTK_POPOVER (self->popup_menu));
 }
 
 static void
index 30b4693078c1682c2a10599b1b1431d7b1a97cc6..0b2801e6335a371bfd346b5528a81d1668905dce 100644 (file)
@@ -376,7 +376,7 @@ gtk_combo_box_size_allocate (GtkWidget *widget,
   gtk_widget_set_size_request (priv->popup_widget,
                                MAX (width, menu_width), -1);
 
-  gtk_native_check_resize (GTK_NATIVE (priv->popup_widget));
+  gtk_popover_present (GTK_POPOVER (priv->popup_widget));
 }
 
 static void
index 32e5c21c088bf5ad6e39cd75b5e4469bd704071a..669305d53944552b77bd8f6c111403c741d923b1 100644 (file)
@@ -383,7 +383,7 @@ gtk_drop_down_size_allocate (GtkWidget *widget,
   gtk_widget_set_size_request (self->popup, width, -1);
   gtk_widget_queue_resize (self->popup);
 
-  gtk_native_check_resize (GTK_NATIVE (self->popup));
+  gtk_popover_present (GTK_POPOVER (self->popup));
 }
 
 static gboolean
index a41c4b49d5c029679d2eeb489e85f67cebbef25b..11bf196e0a5c79f1a4fbc22f111ee1b42a21526a 100644 (file)
@@ -113,7 +113,7 @@ gtk_emoji_chooser_child_size_allocate (GtkWidget *widget,
 
   GTK_WIDGET_CLASS (gtk_emoji_chooser_child_parent_class)->size_allocate (widget, width, height, baseline);
   if (child->variations)
-    gtk_native_check_resize (GTK_NATIVE (child->variations));
+    gtk_popover_present (GTK_POPOVER (child->variations));
 }
 
 static gboolean
index 6c197a45720682901a287cab4497a90547f2ccec..6092e036c5c39871610d9ffa687d2d7819121a07 100644 (file)
@@ -1150,7 +1150,8 @@ _gtk_entry_completion_resize_popup (GtkEntryCompletion *completion)
                                     NULL, FALSE, 0.0, 0.0);
       gtk_tree_path_free (path);
     }
-  gtk_native_check_resize (GTK_NATIVE (completion->popup_window));
+
+  gtk_popover_present (GTK_POPOVER (completion->popup_window));
 }
 
 static void
index dd1cb284cfdcfa0c77a0a76bce2f334be6d4d959..18d4033892eee43a32c637a34fe9540ea8b78643 100644 (file)
@@ -7225,10 +7225,11 @@ gtk_file_chooser_widget_size_allocate (GtkWidget *widget,
   GtkFileChooserWidget *impl = GTK_FILE_CHOOSER_WIDGET (widget);
 
   GTK_WIDGET_CLASS (gtk_file_chooser_widget_parent_class)->size_allocate (widget, width, height, baseline);
+
   if (impl->browse_files_popover)
-    gtk_native_check_resize (GTK_NATIVE (impl->browse_files_popover));
+    gtk_popover_present (GTK_POPOVER (impl->browse_files_popover));
   if (impl->rename_file_popover)
-    gtk_native_check_resize (GTK_NATIVE (impl->rename_file_popover));
+    gtk_popover_present (GTK_POPOVER (impl->rename_file_popover));
 }
 
 static void
index 58c7431814b101060aa65f732e890aaef5d034a0..61fd639a1f2e427d9e0dfa0cf73d466fa657e4d3 100644 (file)
@@ -3276,7 +3276,7 @@ gtk_label_size_allocate (GtkWidget *widget,
     }
 
   if (self->popup_menu)
-    gtk_native_check_resize (GTK_NATIVE (self->popup_menu));
+    gtk_popover_present (GTK_POPOVER (self->popup_menu));
 }
 
 static void
index d0e4037b2fd8f31bfb46d723adc5c10a60900814..483d92ce93571f780348e2b945551a9ef37c93ce 100644 (file)
@@ -77,6 +77,7 @@
 #include "gtklayoutchild.h"
 #include "gtkwidgetprivate.h"
 #include "gtknative.h"
+#include "gtkpopover.h"
 
 #ifdef G_ENABLE_DEBUG
 #define LAYOUT_MANAGER_WARN_NOT_IMPLEMENTED(m,method)   G_STMT_START {  \
@@ -363,7 +364,9 @@ allocate_native_children (GtkWidget *widget)
        child != NULL;
        child = _gtk_widget_get_next_sibling (child))
     {
-      if (GTK_IS_NATIVE (child))
+      if (GTK_IS_POPOVER (child))
+        gtk_popover_present (GTK_POPOVER (child));
+      else if (GTK_IS_NATIVE (child))
         gtk_native_check_resize (GTK_NATIVE (child));
     }
 }
index 93376641c0b70ba1aefb69d099af4baeed4ece7d..d37ee47fc1c389e36f13163f7d07a8e6633c2034 100644 (file)
@@ -319,7 +319,7 @@ gtk_menu_button_size_allocate (GtkWidget *widget,
                             &(GtkAllocation) { 0, 0, width, height },
                             baseline);
   if (self->popover)
-    gtk_native_check_resize (GTK_NATIVE (self->popover));
+    gtk_popover_present (GTK_POPOVER (self->popover));
 }
 
 static gboolean
index b90b748b857c63bcfa084b69a54c51fc8a105f65..41df790cc6b9c7e8008264987218ecd301c878b5 100644 (file)
@@ -4076,10 +4076,10 @@ gtk_places_sidebar_size_allocate (GtkWidget *widget,
                             baseline);
 
   if (sidebar->popover)
-    gtk_native_check_resize (GTK_NATIVE (sidebar->popover));
+    gtk_popover_present (GTK_POPOVER (sidebar->popover));
 
   if (sidebar->rename_popover)
-    gtk_native_check_resize (GTK_NATIVE (sidebar->rename_popover));
+    gtk_popover_present (GTK_POPOVER (sidebar->rename_popover));
 }
 
 static void
index 8d8611bc1c15b581f824082247892adf1a33750b..51af16c465f96b44adc7d448970bcf4de1612c9f 100644 (file)
@@ -36,6 +36,7 @@
 #include "gtkstack.h"
 #include "gtktypebuiltins.h"
 #include "gtknative.h"
+#include "gtkpopover.h"
 #else
 #include <gtk/gtk.h>
 #endif
@@ -319,7 +320,7 @@ gtk_places_view_row_size_allocate (GtkWidget *widget,
 
   GTK_WIDGET_CLASS (gtk_places_view_row_parent_class)->size_allocate (widget, width, height, baseline);
   if (menu)
-    gtk_native_check_resize (GTK_NATIVE (menu));
+    gtk_popover_present (GTK_POPOVER (menu));
 }
 
 static void
index 0aff55995d2dff96b93d4b5c5bb723d176a4ffa4..5d923645df22d2fcbc6b935b210ebbe2f603e561 100644 (file)
@@ -577,10 +577,9 @@ present_popup (GtkPopover *popover)
   return FALSE;
 }
 
-static void
-gtk_popover_native_check_resize (GtkNative *native)
+void
+gtk_popover_present (GtkPopover *popover)
 {
-  GtkPopover *popover = GTK_POPOVER (native);
   GtkWidget *widget = GTK_WIDGET (popover);
 
   if (!_gtk_widget_get_alloc_needed (widget))
@@ -589,6 +588,11 @@ gtk_popover_native_check_resize (GtkNative *native)
     present_popup (popover);
 }
 
+static void
+gtk_popover_native_check_resize (GtkNative *native)
+{
+}
+
 static void
 maybe_request_motion_event (GtkPopover *popover)
 {
index 9cb1208761001362b97db6254ebaea638be93842..1ecb8e57c32c81c060efa4c798de74839b1c73b1 100644 (file)
@@ -121,6 +121,9 @@ GDK_AVAILABLE_IN_ALL
 void gtk_popover_set_default_widget (GtkPopover *popover,
                                      GtkWidget  *widget);
 
+GDK_AVAILABLE_IN_ALL
+void gtk_popover_present (GtkPopover *popover);
+
 
 G_END_DECLS
 
index 447dc5fa658ede2934ecd0d8bef9c74dbb2dad2b..b98489e8cea7370187fa22278e4fe3431a8198e1 100644 (file)
@@ -320,7 +320,7 @@ gtk_popover_menu_bar_item_size_allocate (GtkWidget *widget,
                             &(GtkAllocation) { 0, 0, width, height },
                             baseline);
 
-  gtk_native_check_resize (GTK_NATIVE (item->popover));
+  gtk_popover_present (GTK_POPOVER (item->popover));
 }
 
 static void
index bb39e7f19ba7cee7165419c01d77b178f0d3900d..67c6044fc238df18aaa00ada2c452ca595b932ce 100644 (file)
@@ -1001,5 +1001,5 @@ gtk_scale_button_size_allocate (GtkWidget *widget,
                             &(GtkAllocation) { 0, 0, width, height },
                             baseline);
 
-  gtk_native_check_resize (GTK_NATIVE (priv->dock));
+  gtk_popover_present (GTK_POPOVER (priv->dock));
 }
index 8452344f60e61ab35f3bc6b1ad7fd455a8fa9b65..427bb9a891b673148733f38632a6b1ea423b124d 100644 (file)
@@ -2448,21 +2448,21 @@ gtk_text_size_allocate (GtkWidget *widget,
 
   chooser = g_object_get_data (G_OBJECT (self), "gtk-emoji-chooser");
   if (chooser)
-    gtk_native_check_resize (GTK_NATIVE (chooser));
+    gtk_popover_present (GTK_POPOVER (chooser));
 
   gtk_text_update_handles (self);
 
   if (priv->emoji_completion)
-    gtk_native_check_resize (GTK_NATIVE (priv->emoji_completion));
+    gtk_popover_present (GTK_POPOVER (priv->emoji_completion));
 
   if (priv->magnifier_popover)
-    gtk_native_check_resize (GTK_NATIVE (priv->magnifier_popover));
+    gtk_popover_present (GTK_POPOVER (priv->magnifier_popover));
 
   if (priv->popup_menu)
-    gtk_native_check_resize (GTK_NATIVE (priv->popup_menu));
+    gtk_popover_present (GTK_POPOVER (priv->popup_menu));
 
   if (priv->selection_bubble)
-    gtk_native_check_resize (GTK_NATIVE (priv->selection_bubble));
+    gtk_popover_present (GTK_POPOVER (priv->selection_bubble));
 
   if (priv->text_handles[TEXT_HANDLE_CURSOR])
     gtk_native_check_resize (GTK_NATIVE (priv->text_handles[TEXT_HANDLE_CURSOR]));
index 4816bef439e48e5535cfe4d80a659d523dd8b565..5e263787484c0f9c3cdfe2e16240218a205ba2eb 100644 (file)
@@ -4528,13 +4528,13 @@ gtk_text_view_size_allocate (GtkWidget *widget,
 
   chooser = g_object_get_data (G_OBJECT (text_view), "gtk-emoji-chooser");
   if (chooser)
-    gtk_native_check_resize (GTK_NATIVE (chooser));
+    gtk_popover_present (GTK_POPOVER (chooser));
 
   if (priv->magnifier_popover)
-    gtk_native_check_resize (GTK_NATIVE (priv->magnifier_popover));
+    gtk_popover_present (GTK_POPOVER (priv->magnifier_popover));
 
   if (priv->popup_menu)
-    gtk_native_check_resize (GTK_NATIVE (priv->popup_menu));
+    gtk_popover_present (GTK_POPOVER (priv->popup_menu));
 
   if (priv->text_handles[TEXT_HANDLE_CURSOR])
     gtk_native_check_resize (GTK_NATIVE (priv->text_handles[TEXT_HANDLE_CURSOR]));
@@ -4543,7 +4543,7 @@ gtk_text_view_size_allocate (GtkWidget *widget,
     gtk_native_check_resize (GTK_NATIVE (priv->text_handles[TEXT_HANDLE_SELECTION_BOUND]));
 
   if (priv->selection_bubble)
-    gtk_native_check_resize (GTK_NATIVE (priv->selection_bubble));
+    gtk_popover_present (GTK_POPOVER (priv->selection_bubble));
 }
 
 static void
index def80e57761b2a87ed191e900026ab97a8f87c8a..dda21f82aae46909c5f62c85b6157bd4c6e0141d 100644 (file)
@@ -2654,7 +2654,7 @@ gtk_tree_view_size_allocate (GtkWidget *widget,
     }
 
   if (priv->search_popover)
-    gtk_native_check_resize (GTK_NATIVE (priv->search_popover));
+    gtk_popover_present (GTK_POPOVER (priv->search_popover));
 }
 
 /* Grabs the focus and unsets the GTK_TREE_VIEW_DRAW_KEYFOCUS flag */